Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/add sample agent #6

Merged
merged 19 commits into from
May 24, 2024
Merged

Feat/add sample agent #6

merged 19 commits into from
May 24, 2024

Conversation

Ellenn-A
Copy link
Contributor

@Ellenn-A Ellenn-A commented May 22, 2024

Pull Request

Checklist

  • Have you read nice-fetch-ai-adapter's Code of Conduct? https://github.com/digicatapult/nice-fetch-ai-adapter/.github/blob/main/CODE_OF_CONDUCT.md
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.

PR Type

Please delete options that are irrelevant.

  • Feature
  • Documentation Update

Linked tickets

High level description

Added a fetch.ai sample agent to the repository. Added schemas for DRPC request and response to relevant endpoints.

Detailed description

fetch.ai sample agent has been added to the repository to demonstrate our ability to forward queries received on an endpoint (in particular /send-query endpoint). The query can be forwarded to a Sample Agent from where we would have the ability to communicate with other fetch.ai agents registered on almanach.

Describe alternatives you've considered

We have considered making the Sample Agent a separate repository, but following the fetch.ai's example here we have decided that a simple Sample Agent can exist in the same repository.

Operational impact

Additional context

At the moment the AGENT_ADDRESS is obtained upon startup on the agent and included in config.py

@Ellenn-A Ellenn-A marked this pull request as ready for review May 22, 2024 13:09
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
app/routes/posts.py Outdated Show resolved Hide resolved
app/routes/posts.py Outdated Show resolved Hide resolved
app/routes/posts.py Outdated Show resolved Hide resolved
app/routes/posts.py Outdated Show resolved Hide resolved
app/routes/posts.py Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Ellenn-A Ellenn-A requested a review from msterle May 24, 2024 10:48
Copy link

@msterle msterle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Ellenn-A Ellenn-A merged commit 220920a into main May 24, 2024
4 checks passed
@Ellenn-A Ellenn-A deleted the feat/add-sample-agent branch May 24, 2024 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants